Port "SStechtree startup optimisation" #343
Merged
+13
−16
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
# About the pull request Optimises the initialisation time of the 'Tech Tree' subsystem. Currently, `/datum/controller/subsystem/techtree/Initialize()` is changing the turf of every tile on the tech tree's z-level to `/turf/closed/void`, presumably so that it all appears black rather than space-y. However since it's calling `ChangeTurf()` on the *entire* z-level (300x226, or 67,800 turfs), this is pretty performance intensive. This PR fixes that by just... not changing the turfs, instead only adding a few `/turf/closed/void`s around the edge of the tech tree area so that it still looks the same in-game. This changes the subsystem's init time from 6.4 seconds(ish), to 0.15 seconds(ish). **Before:**  **After:**  Players viewing the tech tree through the console won't notice anything different, as the remaining layer of `/turf/closed/void`s block their view of the rest of the z-level. # Explain why it's good for the game About 6 seconds knocked off of the server's init time. (10%!) *(At least on my end)* # Testing Photographs and Procedure <details> <summary>Screenshots & Videos</summary> <hr> ### Before/after init times from a few different tests: **Before:**    **After:**    <hr> ### Teleporting to it as an observer, there *is* a visible difference: **Before:**  **After:**  <hr> ### But viewing it IC, it all works exactly the same as before: https://github.com/cmss13-devs/cmss13/assets/57483089/14fc7b05-5279-489f-99df-e9acc83b1420 <hr> ### The new `/turf/closed/void` tiles highlighted for reference:  </details> # Changelog :cl: code: Made the Tech Tree subsystem initialise faster. /:cl:
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
About the pull request
Ports this commit:
cmss13-devs/cmss13#5218
Optimises the initialisation time of the 'Tech Tree' subsystem.
Before:
After:
Explain why it's good for the game
Faster init.
Testing Photographs and Procedure
Screenshots & Videos
Put screenshots and videos here with an empty line between the screenshots and the
<details>tags.Changelog
🆑
code: Made the Tech Tree subsystem initialise faster.
/:cl: